-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[IMP] Referrals: Update share jobs #13939
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hi @slinkous - this is ready for a peer review. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few tiny notes. Looks great.
9569c11
to
ea639b3
Compare
Hi @Felicious - this is ready for a final review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @larm-odoo ! just had some optional content suggestions and a required file rename. After that, it's ready to merge!
@robodoo delegate+
ea639b3
to
372053c
Compare
@robodoo r+ |
closes #13939 Signed-off-by: Lara Martini (larm) <larm@odoo.com>
@larm-odoo staging failed: ci/runbot on 0929b461792fc8eedf2511e0f07f3651658ba172 (view more at https://runbot.odoo.com/runbot/build/84910266) |
@robodoo retry |
closes #13939 Signed-off-by: Lara Martini (larm) <larm@odoo.com>
@larm-odoo staging failed: ci/runbot on c6a612ec3817600a1fd3975157c7d7cf5711cbd9 (view more at https://runbot.odoo.com/runbot/build/84912158) |
@robodoo retry |
Updating to current standards.
Original task cardfor this PR.